Conversation
|
Could you please add this file to give type support to typescript apps that uses sqlite-parser? |
|
Hi, @euginio, this is a great idea! I took a look over what you created and there are a few issues. There are three ways to invoke the main parser function. One takes a string and an option options object, and the other two take either a string, an option options, and a callback or just a string and a callback. The latter forms are used to call the parser asynchronously and provide the resulting AST or error to a callback function. There are also a number of other properties defined on the default function, such a a version string, and some additional things that are exported by the module. We probably also want a full set of interfaces or just an any type instead of a partial set of typings for the AST. I believe the current process for releasing 3rd party typings for a library involves releasing them under the |
|
Hi, @nwronski, regarding to your suggestions about issues: I'll make the changes you suggest, but I think we don't need to complete all the API the library exposes in the definition file, but it could be done on-demand. |
|
I think I should go to DefinitelyTyped? |
Declaration file for use sqlite-parser from Typescript code https://basarat.gitbooks.io/typescript/docs/types/ambient/d.ts.html